gtkwindow: default size with fixed size windows

Allow fixed size windows with a default size to grow or shrink as the
content requires, but not smaller than the given default size.

https://bugzilla.gnome.org/show_bug.cgi?id=762974

gtkapplication-dbus: use application id instead of prgname for SM

gnome-session takes an "app id" for client and inhibitor registration,
This app id is supposed to be a desktop file id (complete with the
.desktop extension), but gtk+ currently uses g_get_prgname ().

This commit changes gtkapplication to use the application id instead,
which is a much more natural fit.  gnome-session is going to be updated
to stop using the .desktop extension, too, so everything is consistent.

https://bugzilla.gnome.org/show_bug.cgi?id=763106

gtkplacessidebar: add new other locations with flags signal

We weren't using the open flags on the other locations signal, which
makes impossible for applications like nautilus to act in the same
way that for any other location where the user can choose between
opening in the current view, in a new window or in a new tab.

Add a new signal with an open flags parameter and deprecate
the other-location signal.

https://bugzilla.gnome.org/show_bug.cgi?id=754743

Revert "gtkplacessidebar: add open flags to other locations"

This reverts commit 63a204e1c5b6c since it broke the API.
We will add a new signal instead in an upcomming patch.

https://bugzilla.gnome.org/show_bug.cgi?id=754743

Adwaita: treeview column header scrollbar top border

add a top border in vertical scrollbars in treeviews.

wayland: Don't crash with offscreen windows

Use gdk_window_get_effective_toplevel when looking for
a suitable transient parent, to skip over offscreen windows
that we might encounter in the window tree. This fixes
a crash in glade.

https://bugzilla.gnome.org/show_bug.cgi?id=763110

Adwaita: shrink scroolbars

erroneusly pushed 15px scrollbars with the previous commit, back
to 13px now.

Adwaita: fix a typo

Adwaita: scrollbar style rewrite

works correctly with the new gadgettified GtkRange Cosimo nicelly
delivered, as a bonus steppers are styled for overlay indicator
usage as well.

Updated Russian translation

range: add positional style classes to fill/highlight

Requested by Lapo.

x11: Trap errors around an XRandr call

Apparently, we can get X errors here. Ignore them.
https://bugzilla.gnome.org/show_bug.cgi?id=762907

gtkwindow: windows with a fixed size can shrink

One important aspect of non-resizable windows that we need to preserve
is that they shrink when their content requires less size.

Previous changes to allow the default size to be applied to fixed size
windows would have prevented all fixed size windows from shrinking when
their content requires less size.

Allow shrinking for fixed-size windows unless a default size was
specified.

https://bugzilla.gnome.org/show_bug.cgi?id=762974

Updated Hungarian translation

Adwaita: shrink back scale trough

like it was before the GtkRange gadgettification.

wayland: Clean up key repeat timeout

When returning G_SOURCE_REMOVE from a callback, we need
to reset the source id as well.

gadget: Add more details to warning

All the other warnings in this file include the css node name
and the gadget owners type. Be consistent about this.

gadget: Warn if for_size is too small

It does not make sense to pass a for_size into the size allocation
machinery that is smaller than the min-size in that direction.
Warn if it happens, so we can track it down when it occurs.

Adwaita: really fix scale with marks

added a min-height/width compensation too when in fine-tune mode,
so scale with marks and no labels doesn't make the scale node grow
when in fine-tune.

gtk-launch: add hack to close up dbus-daemon race

If gtk-launch exits before the program it starts fully activates,
then the dbus-daemon may avoid doing the activating method call.

This commit works around the problem by pinging the activated
application, and waiting for a reply.

This is similar to the gvfs-open bug here:

    https://bugzilla.gnome.org/show_bug.cgi?id=746534

https://bugzilla.gnome.org/show_bug.cgi?id=763103

Adwaita: fix a thinko wrt fine-tune scales

added a scale node padding compensation for the trough growth in
fine-tune mode, so now the scale node doesn't grow anymore when
mark labels are present.

icon helper: Properly update on style change

A GtkCssChange of NULL means 'everything changed!'.

entry: Make progress node invisible when appropriate

We were failing to do that, leading to progress not disappearing
anymore after it was initially shown, in the gtk3-widget-factory
entry progress example.

testscale: change example to use upwards labels

There's no example of that in testscale.

Adwaita: fix thinko in fine-tune scale selector

Marks always get class top/bottom, regardless of the scale orientation.

Updated Kazakh translation

Updated Kazakh translation

Check if XRRGetOutputInfo returned a null pointer.

Fixes bug 763023: in certain circumstances, XRRGetOutputInfo will return
a null pointer.  This commit adds a check to detect and handle this
return value.

expander: Reinstate rtl allocation behavior

In rtl mode, we must not just reverse the children,
but also allocate from the right. Use the newly introduced
box gadget api to achieve this.

https://bugzilla.gnome.org/show_bug.cgi?id=762945

check button: Reinstate rtl allocation behavior

In rtl mode, we must not just reverse the children,
but also allocate from the right. Use the newly introduced
box gadget api to achieve this.

https://bugzilla.gnome.org/show_bug.cgi?id=762945

box gadget: Add a way to allocate in reverse

This is needed to preserve expected allocation behavior
in rtl mode.

Updated POTFILES.skip

reftests: add a test for animated box shadow updating clip

This was just fixed by 3e0694284785153944255a0501e84a76c491e4b4

reftests: rename util file frame-inhibitor

We'll use the label-text-shadow-changes-modify-clip utils
more generically, lets name the file after its function.

revealer-size: Fix animation setting

Turn animations off before testing, for all test functions containing
"no_animations".

Updated Greek translation

Remove unused variable

GDK W32: Fix redrawing during drag-move with no composition

revealer: Don't use TRANSITION_NONE for disabled animations

If animations are disabled, the only difference we need to make is that
we don't have to start the transition. Size requests should remain the
same.

https://bugzilla.gnome.org/show_bug.cgi?id=762996

Add GtkRevealer sizing tests

https://bugzilla.gnome.org/show_bug.cgi?id=762996

Adwaita: fix a thinko about toolbar items spacing with margins

gtkwindow: Fix regression with fixed size windows

Previous commit to address the default size introduced a regression
with fixed size windows if no default size was given, the resulting
window would end up much smaller than its actual content.

Updated Korean translation

Updated Korean translation

Adwaita: uniform paddings on image-buttons

fixes https://bugzilla.gnome.org/show_bug.cgi?id=762811

Adwaita: use margin for spacing widgets on toolbars

kind of a workaround, this approach is pretty unmaintainable, I
hope we'll get a CSS property for spacing directly on the parent
container (a toolbar in this particular case).
See https://bugzilla.gnome.org/show_bug.cgi?id=762397

gtkwindow: Use default size even if not resizable

If a window is not resizable (with gtk_window_set_resizable ()),
the size given with gtk_window_set_default_size() is ignored.

The solution to this would be to use gtk_widget_set_size_request() but
that's a GtkWidget API and therefore does not take into account the
client side decorations when in use with GtkWindow.

Refactor the code so that gtk_window_set_default_size() (which is a
GtkWindow API) gives the expected result on non-resizable windows as
well.

bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=762974

W32: Prefer CSD by default

Will make GTK+ more willing to use CSD for all normal windows without
being asked to. Lack of desktop composition will, of course, prevent
it from using CSD (in theory).

GTK_CSD=0 will force CSD to NOT to be used whenever
possible (i.e. in cases where CSD is not specifically requested
by a window, by design).

https://bugzilla.gnome.org/show_bug.cgi?id=759899

GDK W32: Set default cursor from our own theme right away

Otherwise WM-dependent default cursor is used, which does not
match our theme. Worse, later GDK will realize that we have
our own left_ptr cursor and will apply it after all, making
the discrepancy even more noticeable.

https://bugzilla.gnome.org/show_bug.cgi?id=762902

GDK W32: Force correct mouse cursor for custom resize/move

* Explicitly grab the device, setting appropriate cursor on it.
* Fix gdk_device_virtual_set_window_cursor() to just set the
  cursor, without trying to check that mouse is over the given
  window. Also prevent it from immediately resetting cursor.
* Alse take into account things that happen in other parts of
  GDK - don't look for replacement cursor, GDK already did that,
  and don't create a default arrow cursor instead of NULL,
  GDK-W32 already did that up the stack as well.
  Warn about inappropriate cursor == NULL argument instead.

https://bugzilla.gnome.org/show_bug.cgi?id=762711

range: use border allocation for gadget hit test

The border is typically part of the reactive part of the widget. This
matches the pre-gadget behavior.

GDK W32: Implement gdk_win32_screen_get_monitor_workarea()

https://bugzilla.gnome.org/show_bug.cgi?id=763012

Document the .circular style class

scrolledwindow: fix left/right thinko for scrollbar style classes

"left" and "right" were inverted, preumably because the position type
parameter refers to the scrolled window position, and not the scrollbar
itself.

widget: Add brute force method to propagate clip

When a gtk_widget_queue_allocate() on some widget increases the clip,
widget->parent's clip was not updated. This appraoch naively just
unions widget's new clip with widget->parent's clip.

This of course only works if widget and parent share the same GDK
window. In the cases where they don't we can't do anything and need a
better fix.

Fixes label-text-shadow-changes-modify-clip.ui reftest.

Updated Russian translation

Adwaita: more scale style fixes

GDK W32: Use layered windows

Toplevels are now true layered windows that are moved,
resized and repainted via UpdateLayeredWindow() API call.
This achieves transparency without any extra effort,
and prevents window size and window contents desychronization
(bug 761629).

This also changes the way CSD windows are detected. We now
use window decorations to detect CSDiness of a window,
and to decide whether a window should be layered (CSD windows should
be) or not.

Decorations are now stored in the window implementation,
not as a quark-based property of the window-as-gobject.

https://bugzilla.gnome.org/show_bug.cgi?id=748872

GDK W32: custom (non-WM) drag-move and drag-resize code

Normally works only on CSD windows, non-CSD windows continue
to use WM modal loop for drag-resizing and drag-moving. However,
if it is activated on non-CSD windows, it does work.

Has the advantage of being completely immune to AeroSnap.
AeroSnap only worked partially on CSD windows, with the only part
that worked being "don't let users drag window titlebar outside of
the desktop". Now AeroSnap doesn't work on windows moved by
this code at all, which is good, since they currently don't work
well with it due to the way shadows are drawn.

It's possible to also re-implement AeroSnap (or something similar),
but that is a story for another commit.

This code was originally intended to fix the problem of window
size and window contents desynchronization, but failed to achieve
that result in the end. Nevertheless, it serves as a foundation for
other changes to the way window resizing works.

https://bugzilla.gnome.org/show_bug.cgi?id=761629

Adwaita: fix scale with marks size

Updated Spanish translation

Adwaita: scale with marks style fixes

after GtkRange gadgettification the theme needs some adjustments
for the new widget structure.

combobox: Correct the CSS node docs

The box was somehow overlooked in the diagrams.

Add a css node test for comboboxes

Adwaita: fix combos styling

since now the combobox childrens are in a box.

scale: don't add scale label for empty markup

This is a regression from the gadget port.

Updated Lithuanian translation

wayland: synchronize key repeat with server

key repeat is handled client side, which means stalls in the compositor
dispatching key release events can lead to fictious repeat events.

This commit ties key repeat to a server roundtrip to ensure the client
and server are in sync.

https://bugzilla.gnome.org/show_bug.cgi?id=757942

wayland: handle key up events earlier in deliver_key_event

We don't need the key repeat rate or anything like that when
handling key up events, so do key up events first before querying
for that information.

https://bugzilla.gnome.org/show_bug.cgi?id=757942

wayland: make deliver_key_event return void

deliver_key_event is sometimes called from a timeout handler and
sometimes called directly.  We currently erroneously return TRUE
(G_SOURCE_CONTINUE) in the case where it's called directly, but to
no ill effect, since we ignore that return value. In the future,
we're going to need to call it directly in other parts of the code
where the return value would be relevant and handling TRUE, would
require adding redundant code.

Instead, this commit just changes the code to always reset the timer
manually, and never rely on glib's ability to automatically reset
the timer by returning TRUE.  This makes the code smaller, too, since
there's less special casing required.

https://bugzilla.gnome.org/show_bug.cgi?id=757942

win32 theme: Add a naive switch implementation

I couldn't come up with anything better than a button on a button.
But the Windows theme sucks...

win32 theme: Implement horizontal spinbuttons

combobox: Put child into box

Put the combobox child into the box we added for this purpose. That way,
we don't have to maintain the child's position ourselves anymore.

container: Remove check

The check is not working for many containers anymore as they are not the
direct parents of their children. We want to allow this behavior in more
places.

combobox: Put the button in a box

This is so we can put the entry into the box in the next commit.

combobox: Make sure the cellview expands

Windows theme needs the expansion for correct drawing of the focus
outline.

combobox: Ensure an entry gets created for has_entry combos

Previously add/remove on the combobox would result in a cellview being
created.

Updated Galician translation

mir: display->list_devices vfunc was dropped

See 4a6f8a065abace71c0ca32d959ec9b86628bffaa

https://bugzilla.gnome.org/show_bug.cgi?id=762891

Fix last commit

We need to drop the reference to it in configure.ac as well...

MSVC build: Drop crypt implementation for MSVC

We are actually not using the crypt() in GDK-Broadway for quite a while,
so the code implementation for MSVC is actually not used.  So, it's time
to get rid of this.

Updated Hebrew translation

Updated Hebrew translation

MSVC builds: Re-arrange include paths

Search also for the headers in include/gio-win32-2.0, as gio.h will
include those headers at some point, and arrange the include paths in a
dependency hierarchy style, top-to-down.

3.19.11

css node tests: Update notebook results

This is fallout from changing the tab states.

wayland: Check actual impl type in transient loop

If the parent of a transient is not a native Wayland window (e.g.
offscreen window), the transient loop check will crash.

Check for the actual type in the transient loop check and do not assume
the parent is necessarily Wayland native.

bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=761156

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

gdk: Compress window state events

If there are already a window state event for a given window queued
when the window state is changed, drop that event and queue a new event
with a changed_mask based on the state before last event that was queue
without compression.

https://bugzilla.gnome.org/show_bug.cgi?id=762468

progressbar: Don't leave css nodes behind

We create and destroy gadgets inside the hierarchy here,
and if we don't explicitly remove their CSS nodes from the parent,
they stick around.

scale: Don't leave css nodes behind

We create and destroy gadgets inside the scale hierarchy here,
and if we don't explicitly remove their CSS nodes from the parent,
they stick around.

Adwaita: fix outline for scale trough

range: Don't leave css nodes behind

We create and destroy gadgets inside the range hierarchy here,
and if we don't explicitly remove their CSS nodes from the parent,
they stick around.

level bar: Don't leave css nodes behind

We create and destroy gadgets inside the levelbar hierarchy here,
and if we don't explicitly remove their CSS nodes from the parent,
they stick around.

cell renderer: Strip out :drop(active) state

It does not make sense to render every cell in a treeview as
drop target, even when the treeview itself is set as a drop destination.

https://bugzilla.gnome.org/show_bug.cgi?id=761686

Updates

Take a reference on the im context for async ops

This might fix the crash in yelp that some people continue
to see.

https://bugzilla.gnome.org/show_bug.cgi?id=762887

wayland: Restore size when configure size is 0x0

According to xdg_shell, an xdg_surface.configure with size 0x0 should
be interpreted as that it is up to the client to set a size.

When transitioning from maximize or fullscreen state, this means the
client should configure its size back to what it was before being
maximize or fullscreen.

This problem currently only occurs on weston because weston sends a
configure with size 0x0 when transitioning back from maximize or
fullscreen.

bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=762713

Updated Swedish translation

notebook: Tell everyone we reordered stuff

Fixes keybindings for reordering actually emitting signals and properly
updating CSS/the screen.

notebook: Reallocate tab gadgets when we reorder them

Otherwise they stay in their position and don't cause a redraw. Which
really confuses Epiphany users apparently.

https://bugzilla.gnome.org/show_bug.cgi?id=762720